Greptile Summary

This PR hardens two Windows-specific parsing paths: file-index directory skipping is made case-insensitive (fixing missed skips for paths like Node_Modules or .GIT), and UserAssist registry DWORD fields are read as unsigned integers instead of signed, preventing negative focus-time values on long-lived installs. Both fixes are accompanied by focused unit tests.

• scanRules.ts: shouldVisitDir now calls .toLowerCase() before the SKIP_DIRS set lookup; set entries are normalized to lowercase accordingly.
• userAssist.ts: readInt32LE → readUInt32LE for runCount, focusCount, and focusMs, correctly reflecting the Windows DWORD type (values up to ~4.3 billion no longer wrap negative).
• Test helpers in userAssist.test.ts are updated to use writeUInt32LE, and new cases exercise values above INT32_MAX to directly prove the fix.

Confidence Score: 4/5

Both changes are narrow, well-targeted fixes with matching unit tests; the only thing to double-check is the exported SKIP_DIRS constant whose entries are now pre-lowercased.

The unsigned-read change in userAssist.ts is straightforward and the new test directly exercises values that would have produced negatives before. The case-insensitive directory skipping works correctly for all callers of shouldVisitDir. The sole concern is that the exported SKIP_DIRS constant silently changed its entry casing — any future direct consumer calling .has(name) without normalising first would get wrong results, but no such consumer exists today.

scanRules.ts — the exported SKIP_DIRS constant now holds pre-lowercased strings, which is invisible from the type signature.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Directory entry name] --> B["name.toLowerCase()"]
    B --> C{SKIP_DIRS.has?}
    C -- yes --> D[Skip directory]
    C -- no --> E{depth <= MAX_DEPTH?}
    E -- yes --> F[Descend into directory]
    E -- no --> G[Stop recursion]

    H[UserAssist registry blob] --> I["readUInt32LE(OFF_RUN_COUNT)"]
    H --> J["readUInt32LE(OFF_FOCUS_COUNT)"]
    H --> K["readUInt32LE(OFF_FOCUS_MS)"]
    I --> L[runCount: 0..4294967295]
    J --> M[focusCount: 0..4294967295]
    K --> N["focusSeconds = round(focusMs / 1000)"]
    N --> O[Aggregate by friendly app name]
    O --> P[Sort by focusSeconds desc]

_{Reviews (1): Last reviewed commit: "fix(windows): harden file index and User..." | Re-trigger Greptile}